Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support to custom roles #1369

Merged
merged 3 commits into from
Nov 28, 2022

Conversation

SharpEdgeMarshall
Copy link
Contributor

@SharpEdgeMarshall SharpEdgeMarshall commented Nov 17, 2022

Resolves #1183
Resolves #1222
Resolves #1334

@kfcampbell


Behavior

Before the change?

  • custom roles are not working on teams, reporting a change during each plan
  • custom roles are not supported on collaborators

After the change?

  • custom roles are working correctly on teams and collaborators

Other information


Additional info

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)
  • Added the appropriate label for the given change

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes (Please add the Type: Breaking change label)
  • No

Pull request type

Please add the corresponding label for change this PR introduces:

  • Bugfix: Type: Bug
  • Feature/model/API additions: Type: Feature
  • Updates to docs or samples: Type: Documentation
  • Dependencies/code cleanup: Type: Maintenance

@SharpEdgeMarshall
Copy link
Contributor Author

SharpEdgeMarshall commented Nov 23, 2022

@kfcampbell sorry for the insistence but this is the third PR about this issue, I'm trying to bring forward the work of @joshua-hancox that could potentially close 3 issues with a single PR.
You said that you would have been very receptive for reviewing a new PR here
I think this is a really core piece of functionality missing that should be fixed ASAP and I would hope to receive some kind of feedback at least.

@kfcampbell
Copy link
Member

@SharpEdgeMarshall thanks for submitting this! It seems reasonable to me.

Integration tests are passing where passing on main and failing where failing on main. I'll go ahead and merge this PR and create a release for it soon. Hopefully this can put those issues to bed once and for all!

@kfcampbell kfcampbell merged commit b7ac9cb into integrations:main Nov 28, 2022
kazaker pushed a commit to auto1-oss/terraform-provider-github that referenced this pull request Dec 28, 2022
* Add support for custom roles

* Fix doc

* Fix wrong comment
avidspartan1 pushed a commit to avidspartan1/terraform-provider-github that referenced this pull request Feb 5, 2024
* Add support for custom roles

* Fix doc

* Fix wrong comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants